Add an expander reftest

This test check that resizing the window when expanding
the expander yields the same end result as having the
expander expanded to begin with. The test uses the inhibit
mechanism introduced in the previous commit.

Add a way to delay snapshots in reftests

This adds an inhibit api that code from the reftest module
can use to delay the taking of the snapshot. Also refactor
the code in gtk-reftest to use the inhibit mechanism for
its own delaying of the snapshot until after the first
expose.

Forgotten file

Add a reftest for flipping icons

Taking advantage of our new direction flipping ability.

window: Only check for the event widget if clicked on the "content" region

Every button press/release event reaching the the multipress gesture in GtkWindow
and happening in the "title" region must be handled, regardless of the event widget.
Children there wanting the event(s) for themselves are (and were always) expected
to stop event propagation.

So the only place to check for the event widget's "window-dragging" style property
is the "content" region, which matches the pre-gestures behavior.

This fixes some issues with sequences being mistakenly claimed (and events not
propagated further) on situations it shouldn't.

window: Handle manually bubbled events

The multipress gesture must react to either direct events on the
GtkWindow (special cased through _gtk_widget_check_handle_wm_event),
or bubbled events from child widgets. Ensure bubbled events go
through the gesture, those are fed manually to make sure events are
only handled once, in either one or other place. The implicit grab
will ensure that doesn't change mid-action.

window: Refactor _gtk_window_check_handle_wm_event()

Separate the event widget check from event handling

treeview: Ensure the event is consumed on ::row-activate

Otherwise the event is possibly handled, but still propagated further anyway.
Ensure the event is consumed by claiming the current sequence on the
GtkGestureMultiPress::pressed handler.

treeview: restrict the multipress gesture to GDK_BUTTON_PRIMARY

::row-activated only used to be triggered by GDK_BUTTON_PRIMARY, so make
the multipress gesture handling this now to be only triggered by that same
button.

https://bugzilla.gnome.org/show_bug.cgi?id=731020

Revert "Use minimum/natural size semantics"

This reverts commit 6d53c2339f79baa0b295ecc614f41f9daab2e132.

https://bugzilla.gnome.org/show_bug.cgi?id=731054 showed some
major regressions caused by this commit.

Revert "Better resize of expandable columns"

This reverts commit 0050d469b592ec571a5940f1ab1d842a76905e17.

Conflicts:
gtk/gtktreeview.c

Avoid another revert conflict

Moving this bit up a few lines lets the following revert apply
cleanly.

Revert "treeview: Remove unneeded checks"

This reverts commit a158a2aa48c8023f99963642cc2657bff207b82d.

The commit conflicts with a later revert, so move it out of the way.

Add more gestures tests

Some of these sequences look wrong/questionable to me.

gestures test: make more readable

gestures: Add a test for grouping

I'm not entirely convinced the test produces the expected
outcome, here.

Expand gesture tests a big more

Print out sequence state changes, and cancellation, and verify that
claiming a gestures causes the expected state changes.

textview: Only trigger drag gesture on GDK_BUTTON_PRIMARY

This gesture was only meant to react on GDK_BUTTON_PRIMARY (either
through real pointer events, or implicitly assumed from touch events),
as it used to behave before gestures. Otherwise the gtk_drag_begin*()
call assumes being triggered by button 1, and the drag misbehaves
because that button isn't really in the state mask.

https://bugzilla.gnome.org/show_bug.cgi?id=731016

Updated POTFILES.skip

gtk-reftest: Add GMODULE_CFLAGS/LIBS

They aren't included in GTK_DEP_CFLAGS/LIBS so they need to be added
manually.

testsuite: put installed tests in the right place

Follow the prevailing practice and put the installed tests
into $libdir/installed-tests/$PACKAGE instead of
$pkglibdir/installed-tests.

inspector: better type detection for resources

Pass the name to g_content_type_guess. With this, most
CSS gets sniffed as C source code.

gtk-reftest: Don't warn needlessly

Only warn about the --direction argument if the option was
actually specified.

inspector: Go to the right tab when changing objects

When going from attribute mapping to model, it makes most sense
to go directly to the data tab, and when going from an action
name to the owner, we want to show the actions tab. Make it so.

inspector: Allow jumping to actions

The property editor for action-name properties now offers
to go to the object where the action is defined.

inspector: Trivial string change

'Size Groups' looks better than 'Sizegroups'.

reftests: Add reftest for direction change

This reftest makes use of the new feature to add signal handlers.

It adds a libreftest.so module containing all the code for the reftests.
When adding a test named reftest.ui, please keep code contained in a
source file names reftest.c and add that file to Makefile.am.

https://bugzilla.gnome.org/show_bug.cgi?id=730833

Update GtkImage icons when the direction is changed

Don't call gtk_widget_set_state_flags(), it filters the direction flags.
Instead, call gtk_widget_update_state_flags() directly.

https://bugzilla.gnome.org/show_bug.cgi?id=730833

gtk-reftest: Clarify --direction argument parameters

... and print a proper error message when the parameter is not
recognized.

gtk-reftest: Allow loading code for reftests

When connecting signal names, gtk-reftest now allows you to use a colon
in the signal handler name like so:
  module:function_name
where module is a module loaded from the same directory (or the .libs
subdirectory for compatibility with uninstalled libtool) as the running
test and the function is resolved in that module. Of course, normal
function names work as before.

reftests: Add a reftest with signals

reftests: Automatically connect signals

builder: Cleanup

Use the usual way of providing user data to vfuncs:
- Don't allocate it, use the stack
- Use CamelCase for naming the struct

inspector: Handle attribute mapping changes better

When the mapping is changed, notify the property and
queue a draw on the widget. This makes both the inspector
and the app update their display.

inspector: some code reshuffling

inspector: Support saving CSS

We have a file chooser, lets use it.

GtkPrintUnixDialog: Remove some dead code

Automatic template children get cleaned up and NULLed out in destroy,
the g_clear_pointer() calls in finalize never do anything.

gtk: port many nullability annotation fixes from Vala bindings

https://bugzilla.gnome.org/show_bug.cgi?id=730745

Make notebook content redraw during drag

The extra condition here that caused the current child to
not redraw during reordering was introduced in f383e1f1
during the port to ::draw, but was not explained in the
commit message, and removing it has no obvious negative
effect.

https://bugzilla.gnome.org/show_bug.cgi?id=730767

GtkBuilder: Ensure types are fully initialized

Just calling get_type() does not ensure that the signals, properties
and everything else gets set up properly. Ensure it is, by calling
g_type_class_ref() before using the type. This fixes the testcase
added in the previous commit.

Add a failing builder testcase

This shows <signal> failing because class_init hasn't been run
yet.

GtkActionable: Update the docs

Actionable is no longer restricted to "app" and "win" actions.

GtkScrolledWindow: Add some NULL checks back

Turns out these were not quite redundant, gtk_container_forall
may be called very early during construction.

https://bugzilla.gnome.org/show_bug.cgi?id=730924

Revert "GtkMisc: Keep gtk_misc_set_alignment working"

This reverts commit aabc356b80fab025d460f61b6c08743b2c10e20d.

Labels are back to looking at x/yalign (for now).

eventcontroller: Ensure controllers stay alive during event emission

Those might trigger the destruction of some widget that would dispose the
event controller while the event is still being handled, so keep an extra
ref on the controller during event processing.

docs: Add some more information in the "input handling model" chapter

Grabs/Touch/Gestures are now fairly well covered. Only keyboard handling
is left.

gtk: fix several out argument annotations

These mostly just switch from allow-none to optional, nullable, or
both, as necessary.

https://bugzilla.gnome.org/show_bug.cgi?id=730745

gtk: add missing ownership annotations ported from Vala

https://bugzilla.gnome.org/show_bug.cgi?id=730745

gtk: add missing type annotations ported from Vala bindings

https://bugzilla.gnome.org/show_bug.cgi?id=730745

gtk: fix annotation syntax and missing some missing annotations

These changes clean up various errors and omissions resulting from
either slightly incorrect G-I/gtk-doc syntax or missing documentation
blocks.

https://bugzilla.gnome.org/show_bug.cgi?id=730745

gtk: port missing array annotations from Vala bindings

https://bugzilla.gnome.org/show_bug.cgi?id=730745

gtk: fix many callback annotations to include closure information

Without this information introspection-based consumers don't realize
they can include context information, but instead think that they
receive an extra gpointer argument (which they don't know how to
handle).

https://bugzilla.gnome.org/show_bug.cgi?id=730745

Add some gesture tests

This test starts to assemble tests related to event propagation,
interaction with traditional event signal handlers, claiming of
events, etc.

Forgotten file

Oops.

Bump version

Move extract-strings to its own directory

This will hopefully help resolve the circular dependency between
libgtk linking against inspector/libgtkinspector and inspector/
needing extract-strings from gtk/.

I didn't preserve the EXEEXT decorations in this operation -
automake gave me stern warnings about it, so I just dropped them
all. Somebody who cross-builds GTK+ will have to reconstruct this.

Fix distcheck

3.13.2

Don't assert in gtk_event_controller_constructed

This prevents some of our generic object implementation tests
from working with gesture objects. Instead, add g_return_if_fail
checks in all the gesture constructors.

Documentation fixes

Add a few missing symbols, and match up parameters in headers
and doc comments.

wayland: Use event->key.time for setting key event time

We were using event->button.time before. That works because it's part of
the common event header, but it's wrong.

wayland: Remove unused XSERVER_TIME_IS_LATER macro

Silence gtk-doc warnings

Revert "GtkImage: Drop use of GtkMisc::x/yalign"

This reverts commit 54d5b426fca71dcda22ea5467054fa801556c20a.

Conflicts:
gtk/gtkimage.c

Revert "GtkLabel: Drop use of GtkMisc::x/yalign"

This reverts commit b320c198b6d325c35e6f6d32f4f841a913efa956.

Revert "Fix up GtkLabel and GtkImage sizing"

This reverts commit 3b30cfa344ebc035ae7318dda5cfa11b8ab3aee7.

Revert "More label and image sizing fixes"

This reverts commit 68c1e83cf0fd7432dbb151ff351ce225708e3ffb.

window: Do not raise border windows on each size allocation

This prevents these windows from being raised above popovers, which
is about the only child widget that may overlap with those anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=730589

treeview: Make the multipress gesture listen only to bin_window

This gesture is meant to handle row-activated signaling, so make the
gesture ignore events from any other window.

button: Claim the sequence on multipress::pressed

In order to avoid the event from bubbling, as it used to (not) do.

gesture: Fix wrong assert in gtk_gesture_set_window

The window must be part of the event controller widget, so
fix the inverted logic.

eventcontroller: Hide more details

Signals are left hidden, so the only entry point to handle_event/reset
is API.

gtk: Update GtkGesture users to the GtkPropagationPhase semantics change

Mainly doing s/TARGET/BUBBLE/ on the fully ported widgets, but GtkTreeView
where the double click handler has moved to GTK_PHASE_TARGET so it runs
parallelly to the still existing event handlers.

widget: Change GtkCapturePhase behavior in event controllers' handling

Previously, there would be globally just a capture and a bubble phase,
with the event just going down the hierarchy once, and the up once.
GTK_PHASE_TARGET actually meaning "run within event handlers", so in
a hierarchy of 3 widgets, emission would be:

  Capture(C)
  Capture(B)
  Capture(A)
  Target(A) (if event handlers allow)
  Bubble(A)
  Target(B) (if event handlers allow)
  Bubble(B)
  Target(C) (if event handlers allow)
  Bubble(C)

This commit changes this behavior and uses GTK_PHASE_TARGET in a less
misleading way, running only on the widget that was meant to receive
the event. And GTK_PHASE_BUBBLE has taken over the execution place of
GTK_PHASE_TARGET, so the emission remains:

  Capture(C)
  Capture(B)
  Capture(A)
  Target(A)
  Bubble(A) (if event handlers allow)
  Bubble(B) (...)
  Bubble(C) (...)

As it was, GTK_PHASE_BUBBLE was useful for running event controllers
paralelly to event handlers, without modifying a single line in those.
For those mixed scenarios, Any of the other phases will have to be
used at discretion, or the event handlers eventually changed to chain
up and let the default event handlers in GtkWidget to be run.

window: Avoid double event processing in the multipress event controller

The events to those are fed outside the regular event propagation scheme,
through _gtk_window_check_handle_wm_event(), so set the controller to
GTK_PHASE_NONE so events aren't processed first manually, and then
automatically.

gesturezoom: Just return a double in get_scale_delta()

Checking whether the gesture is active is a responsibility of the caller.

gesturerotate: Just return a double in get_angle_delta()

Whether the gesture is active or recognized should be checked before in
callers, or just not used naively.

gesture: Replace gtk_gesture_attach/detach with event controller API

Event controllers now auto-attach, and the GtkCapturePhase only determines
when are events dispatched, but all controllers are managed by the widget wrt
grabs.

All callers have been updated.

gtk: Update callers of GtkGesturePan orientation

eventcontroller: Make the event-mask property and methods private.

This will rarely have any use if no subclassing is allowed, so just make it
private for our own.

gtk: Replace GtkPanOrientation with GtkOrientation

And document GtkOrientation to be more generic. There's little added
value in a separate enum for this.

More label and image sizing fixes

Go back to respecting GtkMisc::xpad/ypad. Not doing so breaks
the misc-alignment reftest. As long as we still derive from
GtkMisc, we may as well do this.

Fix up GtkLabel and GtkImage sizing

When GtkMisc was removed, I forgot to apply the style context
padding. This caused the info-bar-message-types reftest to fail.

Don't use hicolor as the default icon-theme

The icontheme lookup code has a special-case that prefers builtin icons
if the theme name is "hicolor". This is problematic for our reftests,
which run in a barebones environment with not settings.

docs: enum cleanup

Move GtkIMPreeditStyle nad GtkIMStatusStyle to the settings where
they are solely used.

docs: enum cleanup

Move GtkStateType to the GtkWidget apis where it is used, and
mark the enum as deprecated.

docs: remove markup

The overview chapters don't go through shorthand processing, so
don't use them there.

GtkApplication: Fix two muxer-related memory leaks

Drop the ref on the action muxer in finalize, and also make sure
shutdown() tears down the muxer setup done in startup().

When GtkApplication adds itself to a muxer, it causes the muxer to take
a ref on the GtkApplication. This has to be undone in shutdown() to make
sure the GtkApplication doesn't end up holding a ref on itself.

https://bugzilla.gnome.org/show_bug.cgi?id=730383

inspector: Allow changing attribute mappings

docs: Move GdkColor off into a deprecated section

Bump ATK dependency to 2.12

... for the new AtkValue interface that we implement.

Some more updates

Typo fixes

Organize the icons

Sort the icon list in check-icon-theme.c, and add
comments so we have a chance of keeping the order.

Drop an unused object

A downside of using templates is that we can have 'parasitic'
objects in there that don't get used for anything.

GtkIconTheme: Add more debug spew

Still trying to track down test problems in gnome-continuous.

wayland: Add support for show_window_menu

image: Draw the image again

An important line of code was removed in
54d5b426fca71dcda22ea5467054fa801556c20a that lead to an undefined
variable. Add that line back.

combobox: Implement clip

entry: Implement clip